fix: add exit handler to profiler by BioPhoton · Pull Request #1221 · code-pushup/cli

BioPhoton · 2026-01-18T04:59:16Z

Implement performance profiling and trace file generation #1197

Followup:

refactor: use sharded WAL #1231

Note

Introduces a production-ready Node.js profiler with safe shutdown and trace output, and modernizes process-exit utilities.

New NodejsProfiler that auto-creates a WriteAheadLogFile, subscribes to process exit/fatal events, supports sharded trace paths, debug transition markers, stats/flush/filePath, and runtime enable/disable; extensive unit/integration tests and snapshots added
Exit handling API revamped: installExitHandlers replaced by subscribeProcessExit (returns unsubscribe, idempotent, supports exitOnSignal/exitOnFatal/custom exit codes); tests updated
WAL improvements: isLeaderWal/setLeaderWal renamed to isCoordinatorProcess/setCoordinatorProcess; ShardedWal gains coordinator detection via CP_SHARDED_WAL_COORDINATOR_ID; test updates
Docs expanded: profiler features, NodeJSProfiler capabilities, exit handlers, configuration; sink mock extended with getPath
Test utils added (omit-trace-json, awaitObserverCallbackAndFlush) to normalize trace snapshots

^{Written by Cursor Bugbot for commit a6da35d. This will update automatically on new commits. Configure here.}

nx-cloud · 2026-01-18T05:00:32Z

View your CI Pipeline Execution ↗ for commit a6da35d

Command	Status	Duration	Result
`nx run ci:code-pushup -- merge-diffs --files=/h...`	✅ Succeeded	5s	View ↗
`nx run-many --targets=code-pushup --parallel=fa...`	✅ Succeeded	1m 30s	View ↗
`nx run-many --targets=code-pushup --parallel=fa...`	✅ Succeeded	12m 54s	View ↗

☁️ Nx Cloud last updated this comment at 2026-01-27 21:59:42 UTC

pkg-pr-new · 2026-01-18T05:00:47Z

Open in StackBlitz

@code-pushup/ci

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/ci@1221

@code-pushup/cli

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/cli@1221

@code-pushup/core

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/core@1221

@code-pushup/models

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/models@1221

@code-pushup/create-cli

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/create-cli@1221

@code-pushup/nx-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/nx-plugin@1221

@code-pushup/axe-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/axe-plugin@1221

@code-pushup/coverage-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/coverage-plugin@1221

@code-pushup/eslint-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/eslint-plugin@1221

@code-pushup/js-packages-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/js-packages-plugin@1221

@code-pushup/jsdocs-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/jsdocs-plugin@1221

@code-pushup/lighthouse-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/lighthouse-plugin@1221

@code-pushup/typescript-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/typescript-plugin@1221

@code-pushup/utils

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/utils@1221

commit: a6da35d

github-actions · 2026-01-18T05:08:22Z

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit 125089c with previous commit e47bf34.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Categories

🏷️ Category	⭐ Previous score	⭐ Current score	🔄 Score change
Performance	🔴 36	🔴 34
Documentation	🟡 53	🟡 53
Code coverage	🟢 93	🟢 93
Bug prevention	🟡 75	🟡 75
Axe Accessibility	🟡 88	🟡 88
Code style	🟢 100	🟢 100	–
Security	🔴 47	🔴 47	–
Updates	🟡 78	🟡 78	–
Type Safety	🟡 67	🟡 67	–
Miscellaneous	🟡 67	🟡 67	–
Accessibility	🟢 92	🟢 92	–
Best Practices	🟢 100	🟢 100	–
SEO	🟢 92	🟢 92	–

👍 2 groups improved, 👎 1 group regressed, 👍 7 audits improved, 👎 4 audits regressed, 14 audits changed without impacting score

🗃️ Groups

🔌 Plugin	🗃️ Group	⭐ Previous score	⭐ Current score
Lighthouse	Performance	🔴 36	🔴 34
JSDocs coverage	Documentation coverage	🟡 53	🟡 53
Code coverage	Code coverage metrics	🟢 93	🟢 93

31 other groups are unchanged.

🛡️ Audits

🔌 Plugin	🛡️ Audit	📏 Previous value	📏 Current value
Lighthouse	Initial server response time was short	🟩 Root document took 480 ms	🟥 Root document took 610 ms
Lighthouse	Total Blocking Time	🟥 1,530 ms	🟥 2,400 ms
Lighthouse	First Contentful Paint	🟥 3.4 s	🟥 3.2 s
Lighthouse	Speed Index	🟥 7.1 s	🟥 7.3 s
JSDocs coverage	Methods coverage	🟨 11 undocumented methods	🟨 10 undocumented methods
JSDocs coverage	Properties coverage	🟥 38 undocumented properties	🟥 42 undocumented properties
JSDocs coverage	Variables coverage	🟥 273 undocumented variables	🟥 273 undocumented variables
JSDocs coverage	Functions coverage	🟥 582 undocumented functions	🟥 581 undocumented functions
Code coverage	Function coverage	🟩 94.3 %	🟩 94.3 %
Code coverage	Line coverage	🟩 92.7 %	🟩 92.7 %
Code coverage	Branch coverage	🟨 89.2 %	🟨 89.2 %
Lighthouse	Avoids enormous network payloads	🟩 Total size was 2,096 KiB	🟩 Total size was 2,095 KiB
Lighthouse	Minimizes main-thread work	🟥 10.2 s	🟥 9.5 s
Lighthouse	Server Backend Latencies	🟩 1,340 ms	🟩 690 ms
Lighthouse	Time to Interactive	🟥 13.4 s	🟥 13.6 s
Lighthouse	Metrics	🟩 100%	🟩 100%
Lighthouse	Reduce unused CSS	🟥 Potential savings of 125 KiB	🟥 Potential savings of 125 KiB
Lighthouse	Largest Contentful Paint	🟥 11.9 s	🟥 11.7 s
Lighthouse	JavaScript execution time	🟥 4.3 s	🟥 4.1 s
Lighthouse	Uses efficient cache policy on static assets	🟨 31 resources found	🟨 31 resources found
Lighthouse	Remove duplicate modules in JavaScript bundles	🟥 Potential savings of 88 KiB	🟥 Potential savings of 87 KiB
Lighthouse	Max Potential First Input Delay	🟥 1,050 ms	🟥 1,160 ms
Lighthouse	Reduce unused JavaScript	🟥 Potential savings of 226 KiB	🟥 Potential savings of 228 KiB
Lighthouse	Network Round Trip Times	🟩 60 ms	🟩 50 ms
TypeScript	Semantic errors	🟥 36 errors	🟥 37 errors

654 other audits are unchanged.

# Conflicts: # packages/utils/src/lib/exit-process.int.test.ts # packages/utils/src/lib/exit-process.unit.test.ts # packages/utils/src/lib/profiler/profiler.unit.test.ts

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is ON, but it could not run because Privacy Mode (Legacy) is turned on. To enable Bugbot Autofix, switch your privacy mode in the Cursor dashboard.}

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on February 26

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

# Conflicts: # packages/utils/docs/profiler.md # packages/utils/src/lib/profiler/constants.ts # packages/utils/src/lib/profiler/profiler.ts # packages/utils/src/lib/profiler/profiler.unit.test.ts

packages/utils/src/lib/wal.ts

packages/utils/src/lib/profiler/profiler-node.ts

packages/utils/src/lib/wal.ts

github-actions · 2026-01-27T16:43:26Z

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit 125089c with previous commit e47bf34.

💼 Project `utils`

🤨 Code PushUp report has both improvements and regressions.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category	⭐ Previous score	⭐ Current score	🔄 Score change
Documentation	🟡 60	🟡 61
Code coverage	🟢 95	🟢 95

4 other categories are unchanged.

👍 2 groups improved, 👍 5 audits improved, 👎 2 audits regressed

🗃️ Groups

🔌 Plugin	🗃️ Group	⭐ Previous score	⭐ Current score	🔄 Score change
JSDocs coverage	Documentation coverage	🟡 60	🟡 61
Code coverage	Code coverage metrics	🟢 95	🟢 95

13 other groups are unchanged.

🛡️ Audits

🔌 Plugin	🛡️ Audit	📏 Previous value	📏 Current value
JSDocs coverage	Methods coverage	🟨 11 undocumented methods	🟨 10 undocumented methods
JSDocs coverage	Properties coverage	🟥 35 undocumented properties	🟥 39 undocumented properties
JSDocs coverage	Variables coverage	🟥 50 undocumented variables	🟥 50 undocumented variables
JSDocs coverage	Functions coverage	🟥 244 undocumented functions	🟥 243 undocumented functions
Code coverage	Branch coverage	🟩 92 %	🟩 91.9 %
Code coverage	Function coverage	🟩 95.9 %	🟩 96 %
Code coverage	Line coverage	🟩 97.9 %	🟩 98 %

437 other audits are unchanged.

13 other projects are unchanged.

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is ON, but it could not run because Privacy Mode (Legacy) is turned on. To enable Bugbot Autofix, switch your privacy mode in the Cursor dashboard.}

packages/utils/src/lib/wal.ts

packages/utils/docs/profiler.md

packages/utils/src/lib/wal.unit.test.ts

packages/utils/src/lib/profiler/constants.ts

packages/utils/src/lib/wal.ts

packages/utils/src/lib/profiler/profiler-node.ts

packages/utils/src/lib/exit-process.int.test.ts

packages/utils/src/lib/exit-process.unit.test.ts

packages/utils/src/lib/wal.unit.test.ts

packages/utils/src/lib/wal.ts

Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>

fix: add exit handler to profiler

170bab2

github-actions bot assigned BioPhoton Jan 18, 2026

github-actions bot added 📖 Project documentation improvements or additions to the project documentation 🔬 testing writing tests 🧩 utils labels Jan 18, 2026

BioPhoton mentioned this pull request Jan 18, 2026

Implement performance profiling and trace file generation #1197

Open

BioPhoton added 2 commits January 19, 2026 04:46

refactor: impl unsub

6ec0450

refactor: wip

eca6e35

BioPhoton mentioned this pull request Jan 26, 2026

feat(utils): add NodeJS profiler #1219

Merged

BioPhoton added 2 commits January 27, 2026 01:22

Merge branch 'main' into feat/utils/profiler-exit-handler

429fae3

# Conflicts: # packages/utils/src/lib/exit-process.int.test.ts # packages/utils/src/lib/exit-process.unit.test.ts # packages/utils/src/lib/profiler/profiler.unit.test.ts

refactor: wip

56052eb

BioPhoton marked this pull request as ready for review January 27, 2026 00:30

cursor bot reviewed Jan 27, 2026

View reviewed changes

code-pushup deleted a comment from cursor bot Jan 27, 2026

BioPhoton added 2 commits January 27, 2026 15:22

refactor: wip

1dafc5e

Merge branch 'main' into feat/utils/profiler-exit-handler

cee8ccd

# Conflicts: # packages/utils/docs/profiler.md # packages/utils/src/lib/profiler/constants.ts # packages/utils/src/lib/profiler/profiler.ts # packages/utils/src/lib/profiler/profiler.unit.test.ts

cursor bot reviewed Jan 27, 2026

View reviewed changes

packages/utils/src/lib/wal.ts Outdated Show resolved Hide resolved

BioPhoton added 3 commits January 27, 2026 16:40

refactor: split profiler base/node

959dba5

refactor: add exit handler to node profiler

a8d7f3f

refactor: adjust wal defaults

7295a19

cursor bot reviewed Jan 27, 2026

View reviewed changes

BioPhoton added 5 commits January 27, 2026 17:02

refactor: fix unit tests

36a22b8

refactor: fix int tests

5f6e84d

refactor: fix lint

0a36fda

refactor: fix lint

a58c5fa

refactor: fix docs

b11b5fc

refactor: add int tests inc jsonl snapshots

b9bcc72

cursor bot reviewed Jan 27, 2026

View reviewed changes

packages/utils/src/lib/wal.ts Outdated Show resolved Hide resolved

BioPhoton added 3 commits January 27, 2026 18:57

refactor: fix lint

966e3cf

refactor: fix dead code

3ce4ff6

refactor: fix lint

42f450f

BioPhoton requested a review from hanna-skryl January 27, 2026 18:07

hanna-skryl reviewed Jan 27, 2026

View reviewed changes

BioPhoton and others added 8 commits January 27, 2026 20:27

Apply suggestions from code review

a78b157

Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>

refactor: rename method

26b734c

Update packages/utils/src/lib/profiler/profiler-node.ts

0925748

Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>

Update packages/utils/src/lib/exit-process.unit.test.ts

865bad0

Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>

Update packages/utils/src/lib/wal.unit.test.ts

a2577da

Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>

Update packages/utils/src/lib/exit-process.int.test.ts

b62bb86

Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>

refactor: fix format

b9487e7

refactor: fix lint

d320e61

BioPhoton requested a review from hanna-skryl January 27, 2026 19:47

BioPhoton enabled auto-merge (squash) January 27, 2026 19:47

refactor: wip

a6da35d

hanna-skryl approved these changes Jan 27, 2026

View reviewed changes

BioPhoton merged commit 00b97a2 into main Jan 27, 2026
27 of 28 checks passed

BioPhoton deleted the feat/utils/profiler-exit-handler branch January 27, 2026 23:42

Conversation

BioPhoton commented Jan 18, 2026 • edited by cursor bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nx-cloud bot commented Jan 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pkg-pr-new bot commented Jan 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Jan 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code PushUp

🏷️ Categories

🗃️ Groups

🛡️ Audits

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

This is the final PR Bugbot will review for you during this billing cycle

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Jan 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code PushUp

💼 Project utils

🗃️ Groups

🛡️ Audits

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

BioPhoton commented Jan 18, 2026 •

edited by cursor bot

Loading

nx-cloud bot commented Jan 18, 2026 •

edited

Loading

pkg-pr-new bot commented Jan 18, 2026 •

edited

Loading

github-actions bot commented Jan 18, 2026 •

edited

Loading

github-actions bot commented Jan 27, 2026 •

edited

Loading

💼 Project `utils`